Skip to content
This repository has been archived by the owner on Jul 30, 2021. It is now read-only.

Groups changes #49

Merged
merged 13 commits into from
Jun 7, 2020
Merged

Groups changes #49

merged 13 commits into from
Jun 7, 2020

Conversation

RitaOak
Copy link
Collaborator

@RitaOak RitaOak commented Jun 5, 2020

This change is Reviewable

@RitaOak RitaOak requested a review from brecke June 5, 2020 09:18
Copy link
Member

@brecke brecke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 14 of 18 files at r1, 8 of 8 files at r2, 12 of 12 files at r3.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @RitaOak)


src/components/homepage-searchV2.js, line 23 at r2 (raw file):

        </div>
      </div>
      

what is this ………?

@RitaOak RitaOak requested a review from brecke June 7, 2020 22:16
@RitaOak
Copy link
Collaborator Author

RitaOak commented Jun 7, 2020


src/components/homepage-searchV2.js, line 23 at r2 (raw file):

Previously, brecke (Miguel Laginha) wrote…

what is this ………?

I's a different style for the search bar in the v2 homepage, nothing problematic!

@RitaOak RitaOak merged commit 986b146 into oaeproject:master Jun 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants